Skip to content

Comments

Allow also keboola.mcp-server-tool as component#25

Merged
ujovlado merged 1 commit intomainfrom
ujovlado-mcp-server-tool
Sep 3, 2025
Merged

Allow also keboola.mcp-server-tool as component#25
ujovlado merged 1 commit intomainfrom
ujovlado-mcp-server-tool

Conversation

@ujovlado
Copy link
Member

@ujovlado ujovlado commented Sep 2, 2025

Before asking for review make sure that:

  • you created a task for KBC team to update event-schema in Connection (validation in API endpoint)
  • you notified everyone in #general that even schema is changing

This is follow-up to: #24

Diff

Schema

No change, just a copy

Tests

--- tests/events/ext.keboola.mcp-server.tool.json	2025-09-02 22:28:36.645524604 +0200
+++ tests/events/ext.keboola.mcp-server-tool.json	2025-09-02 22:33:49.290221756 +0200
@@ -1,7 +1,7 @@
 {
     "id": 1234567890,
-    "event": "ext.keboola.mcp-server.tool",
-    "component": "keboola.mcp-server.tool",
+    "event": "ext.keboola.mcp-server-tool",
+    "component": "keboola.mcp-server-tool",
     "message": "Test message for mcp-server",
     "description": "",
     "type": "info",

Having this as a component in Developer Portal (already done), we can change the name of a component, which will be reflected in the UI.

Copy link
Contributor

@Matovidlo Matovidlo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ujovlado ujovlado merged commit f1e7575 into main Sep 3, 2025
1 check passed
@ujovlado ujovlado deleted the ujovlado-mcp-server-tool branch September 3, 2025 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants